docs: Add a release note about GtkSizeGroup::ignore-hidden

GtkShortcutsSection: avoid deprecation warnings

We could just as well remove these calls here, since they
have no effect.

Document GtkSizeGroup::ignore-hidden as broken

...and deprecate it, recommending GtkStack as an alternative.

Adwaita: ensure menuitem labels are not overridden

see https://bugzilla.gnome.org/show_bug.cgi?id=767058

entry: Fix undershoot position wrt entry icons

Draw them inside of the icons, which is where the text scrolls.

README: Refer to release notes online

We don't want to maintain this information in two places,
so just link to the online documentation here.

docs: Fix a duplicate ID

This was causing navigation in the online docs to be confused.

Add a warning for a broken situation

When we emit ::draw, the widget should not have alloc_needed set
anymore. If this happens, it indicates a broken situation. Add a
warning to help tracking down why this might occur.

See https://bugzilla.gnome.org/show_bug.cgi?id=765410

gtk3-demo: Fix some css warnings

Update the font: syntax to follow css.

build: add GtkStack accessible private header at the right place

Updated Spanish translation

range: Properly indent closing brace

Make sure it's obvious what if block it belongs to.

Add GtkStackAccessible

Show only the currently visible child to a11y.

tests/scrolling-performance: Define MyTextView

The type is used in widget-factory.ui, so add a dummy definition here to
keep the test working.

Updated Occitan translation

Updated Occitan translation

Adwaita: explicitelly set label caret-color to currentColor

since it doesn't always inherit 'color' color as it should.

Adwaita: make selection on infobars visible

see https://bugzilla.gnome.org/show_bug.cgi?id=745622

Updated Thai translation

Fixes to Catalan translation

Update Catalan translation

Adwaita: remove textview border

see https://bugzilla.gnome.org/show_bug.cgi?id=764203

Updated Thai translation

x11: Replace RROutput with XID as used in GdkX11Monitor

This avoids an unconditional use of XRandr symbol.

placesview: Do not mark icon name as translatable

https://bugzilla.gnome.org/show_bug.cgi?id=766878

3.21.2

Updates

Teach gtk-builder-tool to leave GtkDialog::border-width alone

This is one of the properties that need to be explicitly set.

Updated Portuguese translation

Silence a compiler warning

Initialize the `nearest` variable.

GtkFileChooser: don't override already opened folder

When filechooer is re-used (instead of being created and destroyed
every time it's needed), it might happen that the directory the
application wishes to initially show is the same directory that
was shown last time the filechooser was used. In that case, make sure
filechooser knows that it does not need to do its default "select $pwd
or $home" routine.

https://bugzilla.gnome.org/show_bug.cgi?id=766122

Mention CSS font property in the release notes

Updated German translation

Move release notes to online docs

This will let us expand the information e.g. with code examples,
and can serve as starting material for a future 3->4 migration
guide.

Update migration guide a bit

Resize grips are no more.

Fix a duplicate word in the docs

window: Unfreeze window on unmap

Make sure to keep parity with the number of times we froze the window
when we unmap it.
Otherwise it will permanently stay frozen after being remapped.

https://bugzilla.gnome.org/show_bug.cgi?id=766643
https://bugzilla.mozilla.org/show_bug.cgi?id=1225044

gtktextview: document how to get line space

adds a paragraph to gtk_text_view_get_pixels_below_lines(),
describing how it can be used together with
gtk_text_view_get_pixels_above_lines() to get the total
amount of line spacing between two paragraphs.

https://bugzilla.gnome.org/show_bug.cgi?id=747206

Add a deprecation note

Mark GtkWidget:style property as deprecated in the docs.

Improve CSS docs a bit

Among other things, explain the relationship of background-color
and background-image.

Trivial typo fix

Remove an unused variable

infobar: Update documentation

Remove explicit reference to color from the
gtk_info_bar_set_message_type() docs.

https://bugzilla.gnome.org/show_bug.cgi?id=759037

Updated Norwegian bokmål translation.

gdk: Provide safe fallback for creating GL contexts

If the backend does not override GdkWindowClass.create_gl_context()
then we should provide a fallback that gives an error, instead of
crashing.

https://bugzilla.gnome.org/show_bug.cgi?id=766782

gl: Add a realize() fallback

Calling gdk_gl_context_realize() should always result in a valid result,
so we need to provide a default implementation, to avoid a call to a
NULL function pointer.

Updated Swedish translation

popover: set priv->window before setting doing add_popover() on it

If the popover's relative-to widget is unparented/reparented, we end
up unparenting/reparenting the popover as well. In that case, at the
moment of reparenting, the widget might have been visible (and is
thus mapped again), but priv->window hasn't been set yet.

We must first set priv->window, and then call gtk_window_add_popover(),
that way gtk_popover_map() has its prerequisites straight.

https://bugzilla.gnome.org/show_bug.cgi?id=766323

Typo

Update Arabic translation

stack: Only map children when necessary

gtk_widget_set_parent() will map the widget if the parent is mapped
and the widget is both visible and child-visible. As we currently
only set the child visibility after adding the child, we immediately
map all children that are added to a mapped stack, even when they
are not actually shown. Avoid this by setting the child visibility
before adding the child, so widgets are only mapped when shown.

https://bugzilla.gnome.org/show_bug.cgi?id=766737

Adwaita: add vertical margins to switches on a headerbar

See https://bugzilla.gnome.org/show_bug.cgi?id=766642

Adwaita: shave 1px from the switch

make the switch height and width 1px smaller.

Improve external drives detection

Use g_drive_is_removable for external drives detection. Current heuristic
fails in some cases (e.g. when removable drive is attached before login),
see Bug 765457.

Bump GLib version accordingly.

https://bugzilla.gnome.org/show_bug.cgi?id=765924

gtktextview: fix < public > comment

This was causing problems with the gtk# scanner

treeview: Fix a rubberbanding-related crash

When starting a rubberband selection from an empty area, we could run
into crashes if the selection moves over the rows and then back out
to unpopulated area. Handle this case without crashing.

https://bugzilla.gnome.org/show_bug.cgi?id=766336

scale css node test: Adapt to changes

The style classes are reflected in the output here.

color scale: Sync style classes with GtkScale

We should use the same style classes here, to avoid theme
confusion.

color scale: Really sync style classes with GtkScale

sync the alpha scale styleclass too.

Adwaita: fix scale theming

rely on toplevel styleclass for scale with marks.

See https://bugzilla.gnome.org/show_bug.cgi?id=766440

gtkselectionprivate: minor style fix

This seems to make some issue with the gtk# scanner.

scale: Use more succinct names for style classes

Use .marks-before/after to indicate the presence of marks.
As Lapo points out, compatibility with the previous names
is not really that important, since everything else changed
around it.

https://bugzilla.gnome.org/show_bug.cgi?id=766440

icon theme: Add a warning

Warn about the situation when we've found a resource or file path,
but gdk-pixbuf fails to give us a pixbuf. This generally means that
either pixbuf loaders are not found or the shared-mime database
is missing.

wayland: Survive lack of settings

This can be made to happen eg by setting XDG_DATA_DIRS and
XDG_DATA_HOME to /. Not a useful value, but not a good reason
to crash either.

wayland: Make gdk_wayland_window_get_wl_output() private

There is no need to make it a public API, move it to the private header
instead.

https://bugzilla.gnome.org/show_bug.cgi?id=766566

Updated Slovak translation

wayland: Add get_monitor_at_window to Wayland backend

Given that Wayland has no global coordinate, the only way for gdk to
retrieve the monitor a window last entered is to retrieve it from the
GdkWaylandWindow itself.

Implement the backend specific get_monitor_at_window() to return the
monitor that was last entered by the window.

https://bugzilla.gnome.org/show_bug.cgi?id=766566

wayland: Add API to retrieve the Wayland output

In Wayland, surfaces get an enter/leave notification each time they
enter or leave an output.

Add an API to GdkWaylandWindow to retrieve the output the window has
last entered.

https://bugzilla.gnome.org/show_bug.cgi?id=766566

display: Add vfunc for get_monitor_at_window

Some backends (namely Wayland) do not support global coordinates so
using the window position to determine the monitor will always fail on
such backends.

In such cases, the backend itself might be better suited to identify
the monitor a given window resides on.

Add a vfunc get_monitor_at_window() to the display class so that we can
use the backend to retrieve the monitor, if the backend implements it.

https://bugzilla.gnome.org/show_bug.cgi?id=766566

scale: Bring back marks-related style classes

It turns out that it is too hard (and in some cases, impossible)
to get this information from node positioning, so bring back the
.scale-has-marks-above/below style classes on the main node.

This should allow us to fix the 'pointy sliders'.

https://bugzilla.gnome.org/show_bug.cgi?id=766440

Revert "scrolledwindow: Better size requisition for GTK_SCROLL_NATURAL children"

This reverts commit 096bea3f0ea8e80fe4c9a2a1a860f32c8a94cc33.

This was accidentally pushed.

x11/dnd: Ignore grab broken events from other devices than our DnD device.

The GdkDragContext should only listen to GDK_GRAB_BROKEN events sent to
its own pointer device. It turns out that the passive key grabs mistake
GDK into sending a GdkEventGrabBroken on the master keyboard, which the
DnD machinery mistakes as a signal to cancel the operation.

https://bugzilla.gnome.org/show_bug.cgi?id=766442

wayland: Avoid spurious crossing events from master touch device

Only generate crossing events on wl_touch.down for the virtual master
device used for touch events, and only whenever this virtual device
actually moves across surfaces. This behavior resembles better what is
expected in X11, where the pointer is warped to the touch position
on XITouchBegin.

This avoids the double emission of leave events when the pointer
emulating touch is lifted, that crossing event will be instead
generated when/if the focus surface changes.

https://bugzilla.gnome.org/show_bug.cgi?id=766314

gdk/wayland: use the multi-thread safe wayland API

This is required for proper integration with any other library/application that
may perform wayland API calls and poll() the wayland fd from multiple threads.
Using wl_display_dispatch{_queue}() is thread-safe if not mixed with custom
poll() usage, which GSource/GMainContext does.

Essentially, the problem is that multiple threads polling and reading
the same fd is extremely racy.  Use the wayland provided API for allowing
concurrent access to the wayland display fd.

See the wayland man pages for wl_display_prepare_read(),
wl_display_cancel_read() and wl_display_read_events() for more details.

https://bugzilla.gnome.org/show_bug.cgi?id=763852

scrolledwindow: Better size requisition for GTK_SCROLL_NATURAL children

GtkScrolledWindow leans towards using the minimum size of its child
widget, unless the scrollbar policy is GTK_POLICY_NEVER. This is
probably fine for most GtkScrollable implementations out there.
Especially when using GTK_SCROLL_MINIMUM, which is the default for all
implementations inside gtk+.

However, this is not good for GTK_SCROLL_NATURAL children. eg.,
VteTerminal's minimum size is 1x1 and natural size is the number of
visible rows and columns requested by the user. We really want to use
the natural size unless the user has resized the window to change that.

https://bugzilla.gnome.org/show_bug.cgi?id=766569

Update cursor docs

Include new images for context-menu and for no-drop.

gtk-demo: Update no-drop cursor image

Adwaita is using a different cursor for this now.

gtk3-demo: Add context-menu cursor

The Adwaita cursor theme covers this one now.

scrolledwindow: Remove redundant use of MAX

This code tries to add the minimum content size, if one is set, to the
GtkScrolledWindow's size requisition. This is obvious from the check
for non-negative values of min-content-height and min-content-width.
Using MAX needlessly makes the code harder to read by implying that
there is more to it when there actually isn't.

Fall out from 0d9ebb501df60cf1803858efcd1c79542588abd8

https://bugzilla.gnome.org/show_bug.cgi?id=766569

Updated Kazakh translation
(cherry picked from commit d1585a7a60d760a0c98934f970c4403023a399c3)

scrolledwindow: Fix typo in get_preferred_height calculation

When we are beginning to calculate the height, if the vscrollbar_policy
is not GTK_POLICY_NEVER, and there is no min-content-height, then we
need some small non-zero value to get started. The idea is to always
ask for at least enough to fit the horizontal scrollbar.

Simply put, this should be the mirror image of the corresponding width
calculation code.

Those who got used to the buggy behaviour might notice that their
GtkScrolledWindows are not as tall as they used to be.

Fall out from 55196a705f00564a44647bfc97981db0a783369a

https://bugzilla.gnome.org/show_bug.cgi?id=766530

stack: always update window position at transition start

Sometimes on wayland we're seeing a frame draw before the first tick
callback, where the window position has not yet been updated.

With this change, the window is properly positioned as soon as
gtk_stack_start_transition returns.

https://bugzilla.gnome.org/show_bug.cgi?id=766405

Use upstream gettext

https://bugzilla.gnome.org/show_bug.cgi?id=373745

Updated Spanish translation

Updated Portuguese translation

Updated Portuguese translation

scale: Avoid a crash

gtk_scale_get_layout was not careful enough, leading to a crash
when opening the inspector on a scale.

widget: fix GtkLabelAccessible NULL links.

Fix testsuite/a11y/about.ui GtkAboutDialog :
"CRITICAL **: atk_hyperlink_get_start_index: assertion 'ATK_IS_HYPERLINK (link)' failed"
That is set widget->priv->accessible as soon as accessible object is generated.

When accessible object is created accessible->priv->widget is set,
if widget->priv->accessible is not , then _gtk_label_accessible_update_links
exits early, thus without creating the links on the accessible side.
(This as it checks for the widget to have the accessible set before proceeding).

https://bugzilla.gnome.org/show_bug.cgi?id=766458

widget: No longer postpone style-updated on unrealized widgets

GTK used to not emit GtkWidget::style-updated on widgets that weren't
realized. This sped up construction of complex widgetry in the early
days of GTK3 where we instantly invalidated on every change.
We don't do that anymore, so in theory (and in my limited testing with
widget-factory) this shouldn't be a prolem anymore.

What is a problem though is that postponing style-updated leads to 2
problems:
(1) Unrealized widgets will not emit style-updated which may cause them
    to not properly update their state and return wrong values from
    get_preferred_width/height() etc
(2) Emitting style-updated during realize can happen too late.
    When a widget is not made child-visible by its parent (common
    examples: notebook, paned) it will also not be realized when the
    parent is initially shown. However, when they get realized later
    (after a resize of the parent), they will emit style-updated (and
    potentially queue a resize) during size-allocate.

https://bugzilla.gnome.org/show_bug.cgi?id=765700

scrolledwindow: destroy children in destroy()

If we don't do that, testsuite/gtk/templates starts failing.

gdkwindow: Use GList less

query-settings: Indicate deprecated settings

stack: Don't underallocate the last_child

stack: Remove some code duplication

Remove useless casts from gdk_event_triggers_context_menu calls

query-settings: Don't compare int to uint

listbox: Don't compare int to uint

colorbutton: Stop using state-changed

toolbar: Limit item position to number of contained elements

Greater values don't make much sense.

https://bugzilla.gnome.org/show_bug.cgi?id=682080

wayland: Avoid unitialized memory reads

I didn't pay attention when I replaced g_new0 with g_newa. Oops.